Skip to content

Add "until-found" to HTMLElement.hidden#2254

Merged
github-actions[bot] merged 1 commit into
microsoft:mainfrom
laymonage:add-hidden-until-found
Dec 2, 2025
Merged

Add "until-found" to HTMLElement.hidden#2254
github-actions[bot] merged 1 commit into
microsoft:mainfrom
laymonage:add-hidden-until-found

Conversation

@laymonage

Copy link
Copy Markdown
Contributor

Rebase of #1670

This has now got support from Chrome and Firefox: https://caniuse.com/wf-hidden-until-found

Safari support has been merged: https://webkit.org/b/238266

@github-actions

Copy link
Copy Markdown
Contributor

Thanks for the PR!

This section of the codebase is owned by @saschanaz - if they write a comment saying "LGTM" then it will be merged.

@laymonage
laymonage force-pushed the add-hidden-until-found branch from effecbe to 7bf16e7 Compare November 19, 2025 13:01
@laymonage

This comment was marked as resolved.

@HolgerJeromin

Copy link
Copy Markdown
Contributor

Is the whole block in inputfiles/overridingTypes.jsonc still needed?

@laymonage

Copy link
Copy Markdown
Contributor Author

@HolgerJeromin when I removed it, it produces the following output instead:

hidden: boolean | number | string | null;

which doesn't look accurate.

@HolgerJeromin

Copy link
Copy Markdown
Contributor

Thanks for testing!

@saschanaz

Copy link
Copy Markdown
Contributor

LGTM, thanks!

@github-actions

github-actions Bot commented Dec 1, 2025

Copy link
Copy Markdown
Contributor

There was an issue merging, maybe try again saschanaz. Details

@jakebailey

Copy link
Copy Markdown
Member

@saschanaz Can you try that again?

@saschanaz

Copy link
Copy Markdown
Contributor

LGTM

@github-actions
github-actions Bot merged commit 2afd637 into microsoft:main Dec 2, 2025
6 checks passed
@github-actions

github-actions Bot commented Dec 2, 2025

Copy link
Copy Markdown
Contributor

Merging because @saschanaz is a code-owner of all the changes - thanks!

@saschanaz

saschanaz commented Dec 2, 2025

Copy link
Copy Markdown
Contributor

Yaaaay thanks @jakebailey !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants